Skip to content

chore: remove broken Codecov VS Code extension from recommendations#568

Closed
helmut-hoffer-von-ankershoffen wants to merge 1 commit into
mainfrom
chore/remove-broken-codecov-vscode-extension
Closed

chore: remove broken Codecov VS Code extension from recommendations#568
helmut-hoffer-von-ankershoffen wants to merge 1 commit into
mainfrom
chore/remove-broken-codecov-vscode-extension

Conversation

@helmut-hoffer-von-ankershoffen

@helmut-hoffer-von-ankershoffen helmut-hoffer-von-ankershoffen commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

🛡️ Implements PYSDK-90 following CC-SOP-01 Change Control, part of our ISO 13485-certified QMS | Ketryx Project

Summary

  • Removes codecov.codecov from the recommended VS Code extensions list
  • The Codecov extension is broken and no longer maintained
  • Prevents VS Code from prompting developers to install a non-functional extension

Test plan

  • Verify .vscode/extensions.json no longer lists codecov.codecov
  • Open repo in VS Code and confirm no Codecov extension install prompt appears

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings April 24, 2026 12:04
@helmut-hoffer-von-ankershoffen helmut-hoffer-von-ankershoffen requested a review from a team as a code owner April 24, 2026 12:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes a broken VS Code extension recommendation and introduces a new engineering process guide documenting Jira/Ketryx/GitHub workflows.

Changes:

  • Remove codecov.codecov from .vscode/extensions.json recommendations.
  • Add PROCESS.md documenting bugfix, change request, and release processes.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
PROCESS.md Adds an engineering process guide covering Jira/Ketryx traceability and release workflow steps.
.vscode/extensions.json Updates recommended VS Code extensions by removing the broken Codecov extension.

Comment thread PROCESS.md Outdated
Comment on lines +1 to +3
# Engineering Process Guide

This document describes how the tools and processes used in this repository — GitHub, Jira, Ketryx, and the Confluence-based QMS — relate to each other and how to follow them correctly as an engineer or AI assistant.

Copilot AI Apr 24, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The newly added PROCESS.md file isn’t mentioned in the PR title/description (which says this PR only removes a broken VS Code extension recommendation). Please update the PR metadata to reflect the added engineering process documentation, or split PROCESS.md into a separate PR to keep the change scoped and reviewable.

Copilot uses AI. Check for mistakes.
Comment thread PROCESS.md Outdated

4. Create GitHub PR
- PR description: link to the CR ticket
Example: "Implements PYSDK-CR-XX: https://aignx.atlassian.net/browse/PYSDK-XX"

Copilot AI Apr 24, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The example Change Request link is internally inconsistent: it says Implements PYSDK-CR-XX but the URL points to /browse/PYSDK-XX. This can confuse readers trying to copy/paste the template; align the ticket key in the example text and URL (and consider using a single canonical key format for CRs).

Suggested change
Example: "Implements PYSDK-CR-XX: https://aignx.atlassian.net/browse/PYSDK-XX"
Example: "Implements PYSDK-CR-XX: https://aignx.atlassian.net/browse/PYSDK-CR-XX"

Copilot uses AI. Check for mistakes.
Comment thread .vscode/extensions.json
@codecov

codecov Bot commented Apr 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.
see 6 files with indirect coverage changes

Copilot AI review requested due to automatic review settings April 24, 2026 12:58
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread PROCESS.md Outdated
Comment on lines +1 to +4
# Engineering Process Guide

This document describes how the tools and processes used in this repository — GitHub, Jira, Ketryx, and the Confluence-based QMS — relate to each other and how to follow them correctly as an engineer or AI assistant.

Copilot AI Apr 24, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR title/description indicate this change is only about removing a broken VS Code extension recommendation, but this PR also adds a new 170+ line engineering process document. Please either update the PR title/description to reflect the added process guide, or split this into a separate PR to keep scope and review intent clear.

Copilot uses AI. Check for mistakes.
Comment thread .vscode/extensions.json
Comment on lines 2 to 7
"recommendations": [
"bierner.markdown-mermaid",
"charliermarsh.ruff",
"codecov.codecov",
"daelonsuzuka.nicegui",
"donjayamanne.python-environment-manager",
"fill-labs.dependi",

Copilot AI Apr 24, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the goal is to remove the broken codecov.codecov extension reference, note it is still listed in .devcontainer/devcontainer.json under customizations.vscode.extensions. Consider removing it there too (or documenting why it should remain) so devcontainer users don't still get the broken extension installed.

Copilot uses AI. Check for mistakes.
@helmut-hoffer-von-ankershoffen helmut-hoffer-von-ankershoffen force-pushed the chore/remove-broken-codecov-vscode-extension branch from babadcd to c0858c3 Compare April 24, 2026 13:01
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants